-
Notifications
You must be signed in to change notification settings - Fork 22.7k
Suggest specifying font-size and margin-block on h1 #37827
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Preview URLs External URLs (2)URL:
(comment last updated: 2025-01-29 09:56:16) |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
language and link suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zcorpan couple more quick comments for you. Thanks for making this update!
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
Co-authored-by: Simon Pieters <zcorpan@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM2; thanks @zcorpan!
This regressed in mdn#37827.
See whatwg/html#7867
Description
Add a new section to help developers avoid the special
h1
insection
default styling.Motivation
We're trying to remove the special default styling in the spec and in Firefox. Devtools will have a warning, but the warning should have a link to help developers know what to do and why. My plan is to link to this new section.
Additional details
https://bugzilla.mozilla.org/show_bug.cgi?id=1937568
Related issues and pull requests